Skip to content

DR-001-Arch: Consistent Stack vs. Reference Integration#2560

Open
qor-lb wants to merge 4 commits intomainfrom
lb_arch_dr
Open

DR-001-Arch: Consistent Stack vs. Reference Integration#2560
qor-lb wants to merge 4 commits intomainfrom
lb_arch_dr

Conversation

@qor-lb
Copy link
Contributor

@qor-lb qor-lb commented Feb 7, 2026

Formal decision record based on discussion during last architecture community workshop: https://github.com/orgs/eclipse-score/discussions/1922#discussioncomment-14871055

This intends to establish a discussion basis for further technical dr's such as:

Important

This PR will be kept open at least until 13th Feb before merging to ensure that necessary reviews can be conducted.

@github-actions
Copy link

github-actions bot commented Feb 7, 2026

⚠️ Docs-as-Code version mismatch detected
Please check the CI build logs for details and align the documentation version with the Bazel dependency.

@github-actions
Copy link

github-actions bot commented Feb 7, 2026

The created documentation from the pull request is available at: docu-html

@masc2023
Copy link
Contributor

masc2023 commented Feb 8, 2026

Seems related to #2346, Feature as stand-alone Delivery?

@qor-lb
Copy link
Contributor Author

qor-lb commented Feb 8, 2026

Seems related to #2346, Feature as stand-alone Delivery?

Yes agree, added it to the list in the description above.

PandaeDo
PandaeDo previously approved these changes Feb 9, 2026
Copy link
Contributor

@PandaeDo PandaeDo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Contributor

@opajonk opajonk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

My understanding is the same, at least from the discussion in the last architecture on-site where I was also participating.

(Cross-linking another related DR: #2400)

Copy link
Contributor

@FScholPer FScholPer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What option is accepted or did I miss something?

Copy link
Contributor

@aschemmel-tech aschemmel-tech left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

see inline comment

Copy link
Contributor

@FScholPer FScholPer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

no accepted option defined

@qor-lb
Copy link
Contributor Author

qor-lb commented Feb 12, 2026

Copy link
Contributor

@aschemmel-tech aschemmel-tech left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Original (blocking) remark is covered. As I understood the discussion in "Alignment Tech Leads, Community Leads & Feature Team Leads" meeting, there needs to be some description how we want to deal with "breaking API changes", which would also differentiate between "small" and "big" changes. Maybe a possibility to differentiate is to ask: "is this change affecting the logical interface described in our Feature Architecture?" - because if yes, the process is asking for a "Feature Change Request" which would also result in planning activities.


In particular, the consistent stack approach does not prevent modules or features from being independently buildable or releasable, nor from being used outside the S-CORE stack.

The decision only establishes that, when modules are integrated into S-CORE, their evolution and changes must be justified in terms of stack-level objectives.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Imho the heart of the question is about responsibility for breaking changes and that goes beyond "changes must be justified in terms of stack-level objectives".

Let's assume module X makes a breaking change from version 42 to 43 and an S-CORE release is imminent. Now I see two options how integration can be handled and they somewhat correspond to the considered options 1 and 2:

Option 1: Module X does not integrate, thus we cannot release S-CORE yet. Integrators organize to get the version 43 improvements in.

Option 2: Module X does not integrate, thus we release S-CORE with old version 42. Module X if you want to be in the release, you need to work.

Copy link
Contributor Author

@qor-lb qor-lb Feb 16, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@a-zw from my perspective both options would still be valid even under the current proposed decision. I have added a subsection to the Impact on Governance and Planning section to try and clarify that. Let me know it this resolves your comment or if you feel differently.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants